-
Notifications
You must be signed in to change notification settings - Fork 50
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update swagger generated API crates #590
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #590 +/- ##
=======================================
Coverage ? 57.76%
=======================================
Files ? 168
Lines ? 9912
Branches ? 0
=======================================
Hits ? 5726
Misses ? 4186
Partials ? 0 ☔ View full report in Codecov by Sentry. |
No New Or Fixed Issues Found |
f9b6360
to
9727d46
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like we might be removing some of our custom logic around uuids?
## Type of change ``` - [ ] Bug fix - [ ] New feature development - [x] Tech debt (refactoring, code cleanup, dependency upgrades, etc) - [ ] Build/deploy pipeline (DevOps) - [ ] Other ``` ## Objective Extracted the autogenerated code from #590
Type of change
Objective
Update swagger generator to version 7.2.0, pull in the new templates and generate the new API crates.
I've also updated the build instructions to use .NET 8, as is required in the latest server code.
The version of the server code used to generate this is bitwarden/server@fc1d7c7
I've had to make some small modifications to the server code to get it working,
and I'll link the PR here shortly.bitwarden/server#3760To make this easier to review, I've split this into three commits:
f9b6360: New autogenerated swagger code, no manually edited files here.Moved to a separate PR (Update autogenerated swagger files #593)